Skip to content

Misp objects/#175#176

Closed
haxpak wants to merge 3 commits intoMISP:masterfrom
haxpak:misp-objects/#175
Closed

Misp objects/#175#176
haxpak wants to merge 3 commits intoMISP:masterfrom
haxpak:misp-objects/#175

Conversation

@haxpak
Copy link
Contributor

@haxpak haxpak commented Apr 16, 2019

solves issue #175

"malware-sample": {
"description": "The file itself (binary)",
"ui-priority": 1,
"misp-attribute": "malware-sample"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

malware-sample is definitely correct here, and working: https://github.com/MISP/PyMISP/blob/master/tests/testlive_comprehensive.py#L995

How are you using the object?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was trying to import a text file.
if I place the file as a non malicious plain attachment, it does not render it as an object, which cannot be refrenced with other objects

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you use the API, or the web interface?

@haxpak
Copy link
Contributor Author

haxpak commented Apr 16, 2019 via email

@Rafiot Rafiot self-assigned this Apr 16, 2019
@Rafiot
Copy link
Member

Rafiot commented Apr 18, 2019

Oh, sorry, I misunderstood what you were doing. Right now, MISP always create a simple attribute, not object, for the sane attachments, and you can only create a file object for malwares. I strongly discourage you to change the template that way, because the malware-samples are compressed and encrypted, not the attachments.

@iglocska What do you think? Should we add an object reference "attachment" in the template and always create a file object when we add an attachment to an event?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants